docs: add missing changelog entries for VideoRecorder (#9377) and TreeNode filter fix (#9458)#9474
Conversation
…eNode filter fix (#9458) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Evangelink
added
type/automation
There was a problem hiding this comment.
Pull request overview
Adds two previously-missed entries to the Microsoft.Testing.Platform changelog so the 2.3.0 “UNRELEASED” notes reflect recent merged work.
Changes:
- Added a changelog entry for the new preview
Microsoft.Testing.Extensions.VideoRecorderextension (#9377). - Added a changelog entry for the TreeNode filter dual-filter validation fix and related documentation improvements (
#9458).
Show a summary per file
| File | Description |
|---|---|
| docs/Changelog-Platform.md | Adds missing Added and Fixed bullets for #9377 (VideoRecorder) and #9458 (TreeNode filter validation + docs). |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Low
Evangelink
left a comment
Evangelink
left a comment
There was a problem hiding this comment.
Note
🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.
Expert Review — 22 Dimensions
This is a documentation-only PR (single file: docs/Changelog-Platform.md). All code-level dimensions are N/A.
| # | Dimension | Verdict | Notes |
|---|---|---|---|
| 1 | Algorithmic Correctness | N/A | No code changes |
| 2 | Threading & Concurrency | N/A | No code changes |
| 3 | Security & IPC Contract Safety | N/A | No code changes |
| 4 | Public API & Binary Compatibility | N/A | No code changes |
| 5 | Performance & Allocations | N/A | No code changes |
| 6 | Cross-TFM Compatibility | N/A | No code changes |
| 7 | Resource & IDisposable Management | N/A | No code changes |
| 8 | Defensive Coding at Boundaries | N/A | No code changes |
| 9 | Localization & Resources | N/A | No code changes |
| 10 | Test Isolation | N/A | No test changes |
| 11 | Assertion Quality | N/A | No test changes |
| 12 | Flakiness Patterns | N/A | No test changes |
| 13 | Test Completeness & Coverage | N/A | No code changes |
| 14 | Data-Driven Test Coverage | N/A | No test changes |
| 15 | Code Structure & Simplification | N/A | No code changes |
| 16 | Naming & Conventions | N/A | No code changes |
| 17 | Documentation Accuracy | ✅ LGTM | See notes below |
| 18 | Analyzer & Code Fix Quality | N/A | No src/Analyzers/ changes |
| 19 | IPC Wire Compatibility | N/A | No serialization changes |
| 20 | Build Infrastructure & Dependencies | N/A | No build changes |
| 21 | Scope & PR Discipline | ✅ LGTM | Tightly scoped; correct PR refs |
| 22 | PowerShell Scripting Hygiene | N/A | No .ps1 changes |
Dimensions with findings: 0/22
Dimension 17 — Documentation Accuracy
Both entries are accurate, correctly placed, and well-written:
Added entry (#9377):
- Correctly placed in the
### Addedsub-section of[2.3.0] - UNRELEASED. - Accurately labels the extension as
experimental, includes theffmpegdependency requirement, and names the--capture-videoopt-in flag — all material details a user needs to discover and evaluate this feature. - Attribution (
@Evangelink) and PR link (#9377) are correct.
Fixed entry (#9458):
- Correctly placed in the
### Fixedsub-section of[2.3.0] - UNRELEASED. - Accurately describes both the behavioral change (exit-code improvement from
NotSupportedException→ properInvalidCommandLine) and the documentation improvement (!NOT operator, property-filter semantics). - Attribution and PR link are correct.
One minor observation (not a blocker): the new VideoRecorder entry (#9377) appears after existing entries with higher PR numbers (#9415, #9405, #9426). However, this is a retroactive addition of an omitted entry, and the existing file already has non-strictly-ordered entries (e.g., #9405 currently follows #9415), so the placement is consistent with the file's existing conventions.
Dimension 21 — Scope & PR Discipline
Single-file change, single concern (backfilling two omitted changelog entries), clear PR description with precise cross-references to the source PRs. Nothing to flag.
Overall verdict: ✅ Clean — all 22 dimensions pass. Ready to merge.
Reword to say the mutual exclusion is enforced during command-line validation (ValidateCommandLineOptionsAsync) rather than at CLI parse time, matching the actual behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Evangelink
deleted the
adhoc-qa/2026-06-27-changelog-fixes-db90180043c93528
branch
Two changelog entries were omitted from
docs/Changelog-Platform.mdwhen their PRs were merged on 2026-06-26:Added:Microsoft.Testing.Extensions.VideoRecorder(#9377) — new preview extension for screen recording during test runs.Fixed: TreeNode filter--treenode-filter+--filter-uidmutual exclusion moved to proper CLI validation time (#9458), plus documentation improvements for the!NOT operator.Both entries are added to the
## [2.3.0] - UNRELEASEDsection in the correct (Added/Fixed) subsections.Generated by the Adhoc QA workflow.
Add this agentic workflows to your repo
To install this agentic workflow, run